Skip to content

POC: Real time updates #19372

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

POC: Real time updates #19372

wants to merge 10 commits into from

Conversation

madsrasmussen
Copy link
Contributor

@madsrasmussen madsrasmussen commented May 20, 2025

Connecting to the Signal R hub introduced in this PR:
#17776

Discoveries while building this POC:

The created event should include the parent key. Or we need a separate event with this information
Currently, the created event includes the key of the newly created item. For the UI to update correctly, we also need to know the parent so we understand where to load from.

We need to include a "session key" to exclude SignalR events from the current user session.
We already have an internal event system that keeps track of UI updates within the current session. Updates will happen twice when SignalR events are also returned.

Server events should include the user key of the person who performed the action.
In the UI, we would like to display which user performed an action.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant